Skip to content

Fixing static checks in OL adapter on main#60042

Merged
amoghrajesh merged 1 commit intoapache:mainfrom
amoghrajesh:ol-static-checks
Jan 2, 2026
Merged

Fixing static checks in OL adapter on main#60042
amoghrajesh merged 1 commit intoapache:mainfrom
amoghrajesh:ol-static-checks

Conversation

@amoghrajesh
Copy link
Contributor

@amoghrajesh amoghrajesh commented Jan 2, 2026 •
edited
Loading

Main is failing with this error:

diff --git a/providers/openlineage/src/airflow/providers/openlineage/plugins/adapter.py b/providers/openlineage/src/airflow/providers/openlineage/plugins/adapter.py
index 7640140..1ed20b7 100644
--- a/providers/openlineage/src/airflow/providers/openlineage/plugins/adapter.py
+++ b/providers/openlineage/src/airflow/providers/openlineage/plugins/adapter.py
@@ -36,7 +36,7 @@ from openlineage.client.facet_v2 import (
 )
 from openlineage.client.uuid import generate_static_uuid
 
-from airflow.providers.common.compat.sdk import conf as airflow_conf, Stats
+from airflow.providers.common.compat.sdk import Stats, conf as airflow_conf
 from airflow.providers.openlineage import __version__ as OPENLINEAGE_PROVIDER_VERSION, conf
 from airflow.providers.openlineage.utils.utils import (
     OpenLineageRedactor,

Fixing it. Caused by #60026 which was merged too soon

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh
Copy link
Contributor Author

amoghrajesh commented Jan 2, 2026

Failing checks unrelated and are already being fixed in parallel

@amoghrajesh amoghrajesh merged commit 74192f5 into apache:main Jan 2, 2026
85 of 87 checks passed
arjav1528 pushed a commit to arjav1528/airflow that referenced this pull request Jan 2, 2026
@amoghrajesh @arjav1528
Fixing static checks in OL adapter on main (apache#60042)
08d5843
henry3260 pushed a commit to henry3260/airflow that referenced this pull request Jan 3, 2026
@amoghrajesh @henry3260
Fixing static checks in OL adapter on main (apache#60042)
49cd1e5
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
@amoghrajesh @stegololz
Fixing static checks in OL adapter on main (apache#60042)
0407a69
@jscheffl jscheffl mentioned this pull request Jan 14, 2026
Closed
jhgoebbert pushed a commit to jhgoebbert/airflow_Owen-CH-Leung that referenced this pull request Feb 8, 2026
@amoghrajesh @jhgoebbert
Fixing static checks in OL adapter on main (apache#60042)
d3d5c69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vatsrahul1001 vatsrahul1001 vatsrahul1001 approved these changes

@mobuchowski mobuchowski Awaiting requested review from mobuchowski mobuchowski is a code owner

@potiuk potiuk Awaiting requested review from potiuk

@shahar1 shahar1 Awaiting requested review from shahar1

@jscheffl jscheffl Awaiting requested review from jscheffl

Assignees

No one assigned

Labels

area:providers provider:openlineage AIP-53

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@amoghrajesh @vatsrahul1001